Conversation
|
이거 PR 리뷰중인데 이전 PR 링크주신 부분에 올려주신 화면 영상이 현재 PR의 구현부를 전부 보여주지 못하는 것 같아서, 혹시 영상 새로 찍어서 올려주시거나 그게 어려우시다면 그냥 구현하신거 싹다 한 PR에 다시 적어서 영상 2~3개 정도로 나눠서 PR 올려주시면 리뷰하기 좋을것 같아요! |
…to design/#38-design-mypage-ui
AIsiteru98
left a comment
AIsiteru98
left a comment
There was a problem hiding this comment.
뷰모델은 안올린건가?
그리고 주요 코드 설명 있으면 더 좋을둣
0Hooni
left a comment
0Hooni
left a comment
There was a problem hiding this comment.
Navigation 관련 코멘트 꼭 확인 하신뒤 개선이 필요해보입니다!
필요하시다면 같이 고민해도 좋습니다👍
|
|
||
| HStack(spacing: 12) { // 버튼 | ||
| NavigationLink( // 로그아웃 | ||
| destination: OnboardingView().navigationBarBackButtonHidden() |
There was a problem hiding this comment.
이렇게 초기 로그인뷰로 가게 되면 그 전까지 쌓은 뷰가 다시 Navigation에 동일하게 쌓이게 되지 않을까 싶어요!
성능적이나 메모리적으로 한번 확인을 거쳐보면 좋을것 같아보이네요
There was a problem hiding this comment.
@youz2me 예전 회의때 잠깐 얘기를 했었는데, 혹시 좋은 방법이 있을까요?
There was a problem hiding this comment.
https://stackoverflow.com/questions/57334455/how-can-i-pop-to-the-root-view-using-swiftui
쌓여있던 네비게이션 뷰를 전부 pop해주면 될 것 같아요 ~ 자세한 방법이 위에 나와있어서 가져왔습니다! 한번 참고해보셔용
There was a problem hiding this comment.
youz2me
left a comment
youz2me
left a comment
There was a problem hiding this comment.
고생하셨어요 ~~ 충돌난 부분 풀받아서 해결하고 푸시하셔야 할 것 같아서 Request Change로 달아둘게요!
|
|
||
| HStack(spacing: 12) { // 버튼 | ||
| NavigationLink( // 로그아웃 | ||
| destination: OnboardingView().navigationBarBackButtonHidden() |
There was a problem hiding this comment.
이부분은 굳이 줄바꿈 안해도 괜찮지 않을까요..? 혹시 Lint쪽에서 오류나는거면 이야기해주세요!
There was a problem hiding this comment.
수정했습니당 Lint에서도 오류 안나요!
There was a problem hiding this comment.
이렇게 중복되는 버튼의 경우에는 ViewModifier 사용해서 코드 간결하게 유지해도 괜찮을 것 같아요 ~~
There was a problem hiding this comment.
ViewModifier 사용하는걸로 바꿨습니당
There was a problem hiding this comment.
Image(.iconMyHeartFill)로 좀 더 간편하게 사용할 수 있어요 ~~ 자체 자동완성도 해주니까 사용해보시면 좋을 것 같습니다
There was a problem hiding this comment.
이걸 몰랐네.... 다른 파일들도 가능한건 다 수정했습니다~! 꿀팁 감사해용
| .foregroundColor(Color.gray10) | ||
| .font(.sCoreDream(.bold, size: 16)) | ||
|
|
||
| Text(userViewModel.openNow ? "영업 중" : "휴무") |
UserViewModel추가, Color.삭제, 이미지 수정, 정렬 버튼 ViewModifier 적용
Develop 변경사항 pull 받아서 수정
프로젝트 파일 깨짐 수정, 기존 파일 추가
4 participants
🔥 PR 타입 (하나 이상 선택)
🔗 연결된 이슈
📄 작업 내용
👀 기타 더 이야기해볼 점